Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

style with symbols #79

Open
wants to merge 209 commits into
base: main
Choose a base branch
from
Open

style with symbols #79

wants to merge 209 commits into from

Conversation

Nicolasribot
Copy link

@Nicolasribot Nicolasribot commented Feb 7, 2022

A cleaning of a work started some years ago to support a style displaying symbols.
Symbol icons are based on OSM icons converted from SVG.
This style can be combined with other styles to add icons.
WIP: labels at z18, more properties should be defined in generate_styles.py to control symbols behaviour. Some symbols are missing (parking built from ways for instance).
Some examples of the style: https://github.com/Nicolasribot/basemaps/blob/master/README.md

user='osm',
host='docker.for.mac.localhost',
port=5436,
user='nicolas',
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

could you revert this change?

@@ -0,0 +1,113 @@
#define PASTER(str) #str
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This base mapfile is not linked to the symbols support, isn't?

@yjacolin
Copy link
Collaborator

yjacolin commented Feb 9, 2022

@Nicolasribot thank you for you huge work on this. Unfortunatly this PR is too big and we can review it correctly. Could you split it in few part? Scripts, topobase and symbols at least? My browser failed to display diff file after "scripts".

@Nicolasribot
Copy link
Author

Yes I will make the changes.

andreassteffens and others added 14 commits April 28, 2022 10:46
…eration of GetCapabilities document

[refactor] homogenized template map file indentation
Added new style for displaying housenumbers & sped up generation of GetCapabilties document
…'s clean target so they will be properly recreated by calling 'make clean && make'
…nal database view

[fix] added missing placeholders for Makefile variables
Optimized rendering performance of symbol layers
…attached to building objects

[refactor] hardened SQL statements in post-symbols.sql.in
[refactor] optimized database structure for faster queries on symbol and housenumber tables
…bility and causing massive performance issue on lower zoom levels
included address information attached to buildings in imposm import mapping
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants